NMS-19652: Update documentation for trapd config#8411
Merged
Shahbaz-dataq merged 56 commits intofeatures/trapd-config-db-migrationfrom Apr 7, 2026
Merged
NMS-19652: Update documentation for trapd config#8411Shahbaz-dataq merged 56 commits intofeatures/trapd-config-db-migrationfrom
Shahbaz-dataq merged 56 commits intofeatures/trapd-config-db-migrationfrom
Conversation
…-19611-trapd-upload-get-and-update-rest-endpoint
…-19621-integrate-trapd-UI-endpoints
…-19621-integrate-trapd-UI-endpoints
…-19621-integrate-trapd-UI-endpoints
…-19621-integrate-trapd-UI-endpoints
…-19621-integrate-trapd-UI-endpoints
…-19652-trapd-documentation
…-19652-trapd-documentation
…-19652-trapd-documentation
Base automatically changed from
jira/NMS-19621-integrate-trapd-UI-endpoints
to
features/trapd-config-db-migration
April 6, 2026 16:13
…-19652-trapd-documentation
synqotik
requested changes
Apr 6, 2026
Contributor
synqotik
left a comment
synqotik
left a comment
There was a problem hiding this comment.
Looks good! Just one small fix.
| = Trapd REST API (v2) | ||
| :description: Manage trapd configuration through the OpenNMS REST v2 API. | ||
|
|
||
| You can read and update trapd configuration over REST without editing `trapd-configuration.xml` manually. |
Contributor
There was a problem hiding this comment.
Should change line 5 to just:
You can view and update trapd configuration over REST.
I think we want to downplay that there's even an XML file at all.
Shahbaz-dataq
merged commit af749fc
into
features/trapd-config-db-migration
5 checks passed
cgorantla
added a commit
that referenced
this pull request
Apr 22, 2026
* NMS-19448: Trapd Config to DB: Migrate trapd-configuration.xml to * NMS-19952: Create initial scaffolding for UI (#8345) * initial changes * general configuration ui * changes * Changes * NMS-19597: Create form for SNMP v3 User creation and edit (#8351) * create user * fixes * fixes * revert changes * changes * new changes * changes * NMS-19611: Expose rest endpoint for upload/update trapd configuration (#8363) * upload, get and update rest endpoints * changes for dto * test cases * new rest endpoints or snmp v3 users * mappers fixes * field use address from varbind fix * changes * new fixes * new changes * test cases fix * test fix * checkes reverted * user updates changes * unused code removed * menu item * review changes * response messages update * DAO update fix * NMS-19621: Integrate get/upload/update rest endpoint for trapd config (#8370) * upload, get and update rest endpoints * changes for dto * test cases * new rest endpoints or snmp v3 users * mappers fixes * field use address from varbind fix * Trapd UI rest endpoint integration * new changes * new changes * changes * changes * changes * change * search changes * changes * new fixes * new changes * test cases fix * test fix * api changes * checkes reverted * new changes for fe * changes * test coverage * user updates changes * scv changes and test coverage * unused code removed * menu item * route fix * test fixes * naming fixes * unused imports * review changes * response messages update * validator fix * DAO update fix * DAO fixes * changelog * New changes and test fixes * new fixes * test fix * NMS-19652: Update documentation for trapd config (#8411) * upload, get and update rest endpoints * changes for dto * test cases * new rest endpoints or snmp v3 users * mappers fixes * field use address from varbind fix * Trapd UI rest endpoint integration * new changes * new changes * changes * changes * changes * change * search changes * changes * new fixes * new changes * test cases fix * test fix * api changes * checkes reverted * new changes for fe * changes * test coverage * user updates changes * scv changes and test coverage * unused code removed * menu item * route fix * test fixes * naming fixes * unused imports * review changes * trapd documentation * response messages update * validator fix * DAO update fix * config dao fix * DAO fixes * changelog * changes * menu code revert * review comment * NMS-19128: Removing trapd-configuration.xml from ConfigTester * NMS-19128: Fix few issues with UI integration fix SHA2** are getting rejected remove general config and note Allow snmp-trap-address to be optional * NMS-19128: Restrict access to admin/provision roles * NMS-19125: Fix reload daemon for trapd * NMS-19128: Few more validations * NMS-19128: remove unused imports and fix spelling * NMS-19128: Move trapdConfigDao to application context dao This will allow users to disable trapd * NMS-19218: Explicitly disable UsmUserTable logging This will prevent leaking of user authentication details * NMS-19128: passphrases need to be at least 8 characters --------- Co-authored-by: Shahbaz Masood <159028986+Shahbaz-dataq@users.noreply.github.com> Co-authored-by: Shahbaz <rmasood@nanthealth.com> Co-authored-by: Chandra Gorantla <chandra@opennms.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
External References